Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/config events #626

Merged
merged 19 commits into from
Feb 11, 2025
Merged

Feat/config events #626

merged 19 commits into from
Feb 11, 2025

Conversation

tonai
Copy link
Collaborator

@tonai tonai commented Jan 24, 2025

Corrige #618
Il manque encore:

  • integration texte riche pour le détail
  • flux RSS
  • on n'affiche pas le lien pour aller sur la page de configuration, mais il n'y a pas d'affichage de page 403 si on rentre manuellement l'URL et que l'on a pas les bons droits (pas accès à l'entrepôt ou pas le droit annexe) => nécessite un rework global pour résoudre ce point

Autres points:

  • utilise cache: "no-store" pour charger le fichiers des alertes et éviter d'utiliser le cache navigateur qui pose problème
  • actuellement on affiche le menu de l'entrepot "cartes.gouv.fr-config" pour cette page

@tonai tonai requested a review from ocruze January 24, 2025 15:48
@tonai tonai force-pushed the feat/config-events branch from a35ac10 to 100a8fd Compare January 30, 2025 11:49
@tonai tonai force-pushed the feat/config-events branch 2 times, most recently from 8d52395 to 665db4c Compare February 6, 2025 16:33
Copy link
Contributor

@slafayIGN slafayIGN left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci, ça fonctionne super bien !

Je te demande toutefois quelques renommages sur les routes pour les avoir en français comme sur le reste du site et une réorganisation de la page niveau de service pour mieux en hiérarchiser le contenu et avoir l'accès au RSS.

assets/entrepot/pages/config/Alerts.locale.tsx Outdated Show resolved Hide resolved
assets/pages/assistance/ServiceStatus.tsx Outdated Show resolved Hide resolved
assets/router/router.ts Outdated Show resolved Hide resolved
src/Controller/RSSFeedController.php Outdated Show resolved Hide resolved
@slafayIGN
Copy link
Contributor

Aussi lorsque les titres des alertes sont un peu long (ce qui est facilement le cas), le tableau a un scroll horizontal et les boutons de modification et suppression se retrouvent cachés.
image

@slafayIGN
Copy link
Contributor

Le layout de la page Niveau de service est encore incomplet, j'ai marqué unresolved mon commentaire (qui était un peu long)

h1. Niveau de service

h2. Alertes, incidents et maintenances programmées

Bouton secondaire ou moins important avec icône : Suivre par RSS

Les alertes ou s'il n'y en a pas un unique paragraphe "Aucune alerte en cours."

h2. Disponibilité des services

p. Le tableau de ci-après, basé sur l’outil Uptrends, indique la disponibilité de tous les services de la Géoplateforme sur les 30 derniers jours.

iframe uptrends

@tonai tonai force-pushed the feat/config-events branch from 8032709 to c0219b5 Compare February 7, 2025 17:16
@ocruze ocruze merged commit f6c4ea5 into main Feb 11, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants